-
Notifications
You must be signed in to change notification settings - Fork 18
[APO-2825] Add codegen test for state variables in inline subworkflow nodes #3646
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[APO-2825] Add codegen test for state variables in inline subworkflow nodes #3646
Conversation
… nodes - Add stateVariables field to InlineSubworkflowNodeData type - Update InlineSubworkflowNodeDataSerializer to handle stateVariables - Update inline-subworkflow-node.ts to use state variables from node data - Create test fixture that verifies both parent and inner state files are generated Co-Authored-By: vargas@vellum.ai <vargas@vellum.ai>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
| " | ||
| `; | ||
|
|
||
| exports[`generateCode > should generate code for %1 inline-subworkflow-with-state-variables.ts > nodes/subworkflow_with_state/state.py 1`] = ` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ApolloBot followup this PR to do the same thing but for inline map nodes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linear ticket created: https://linear.app/vellum/issue/APO-2830/add-state-variables-support-for-inline-map-nodes-in-python-sdk
Devin session: https://app.devin.ai/sessions/8772b621c9194ddfa583b17c6156d6a7
Built on Vellum
| " | ||
| `; | ||
|
|
||
| exports[`generateCode > should generate code for %1 inline-subworkflow-with-state-variables.ts > state.py 1`] = ` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ApolloBot followup this PR to do the same thing but for inline subworkflow tools within Agent Nodes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linear ticket created: https://linear.app/vellum/issue/APO-2831/add-state-variables-support-for-inline-subworkflow-tools-within-agent
Devin session: https://app.devin.ai/sessions/dffea20100064781a6b75e21690b9f56
Built on Vellum
Adds support for state variables in inline subworkflow nodes by adding the
stateVariablesfield toInlineSubworkflowNodeDatatype/serializer and updating the generator to use it instead of hardcoding an empty array. This prevents state files from "disappearing" in nested subworkflow scenarios.